Skip to content

Conversation

@martinflorian-da
Copy link
Contributor

Fixes https://github.com/DACH-NY/cn-test-failures/issues/3885 (Integration test participant OutOfMemory encountered: Java heap space)

I didn't touch the initial heap size for now; we only reach it rarely so seems to be more economical this way.

Pull Request Checklist

Cluster Testing

  • If a cluster test is required, comment /cluster_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If a hard-migration test is required (from the latest release), comment /hdm_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.

PR Guidelines

  • Include any change that might be observable by our partners or affect their deployment in the release notes.
  • Specify fixed issues with Fixes #n, and mention issues worked on using #n
  • Include a screenshot for frontend-related PRs - see README or use your favorite screenshot tool

Merge Guidelines

  • Make the git commit message look sensible when squash-merging on GitHub (most likely: just copy your PR description).

@martinflorian-da martinflorian-da requested a review from nicu-da June 11, 2025 11:37
@martinflorian-da martinflorian-da marked this pull request as ready for review June 11, 2025 11:38
Fixes DACH-NY/cn-test-failures#3885
(Integration test participant `OutOfMemory encountered: Java heap space`)

I didn't touch the initial heap size for now; we only reach it rarely so seems to be more economical this way.

[ci]

Signed-off-by: Martin Florian <[email protected]>
@martinflorian-da martinflorian-da force-pushed the martinflorian-da/cntf-3885-bump-heap-space branch from ab9c1e1 to 83db6d7 Compare June 11, 2025 11:41
Copy link
Contributor

@nicu-da nicu-da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EAT ALL THE MEMORY!
Download more ram in progress 🎉

@martinflorian-da martinflorian-da merged commit 82c3c88 into main Jun 11, 2025
60 checks passed
@martinflorian-da martinflorian-da deleted the martinflorian-da/cntf-3885-bump-heap-space branch June 11, 2025 12:43
martinflorian-da added a commit that referenced this pull request Aug 4, 2025
Fixes DACH-NY/cn-test-failures#5313

The amount of the increase is inspired by what we did in #1075

Also bumping the memory constraint on the container by the same amount for good measure; may I be forgiven the awkward final number there.

[ci]

Signed-off-by: Martin Florian <[email protected]>
martinflorian-da added a commit that referenced this pull request Aug 4, 2025
Fixes DACH-NY/cn-test-failures#5313

The amount of the increase is inspired by what we did in #1075

Also bumping the memory constraint on the container by the same amount for good measure; may I be forgiven the awkward final number there.

[ci]

Signed-off-by: Martin Florian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants